Skip to content

Storage Cleaner Precompile#1224

Merged
boundless-forest merged 35 commits intopolkadot-evm:masterfrom
moonbeam-foundation:ahmad-clear-suicided-storage
Apr 3, 2024
Merged

Storage Cleaner Precompile#1224
boundless-forest merged 35 commits intopolkadot-evm:masterfrom
moonbeam-foundation:ahmad-clear-suicided-storage

Conversation

@ahmadkaouk
Copy link
Copy Markdown
Contributor

@ahmadkaouk ahmadkaouk commented Oct 16, 2023

Adds a precompile that enable to delete the storage entries of contracts marked as Suicided

@ahmadkaouk ahmadkaouk marked this pull request as ready for review January 29, 2024 12:50
@ahmadkaouk ahmadkaouk requested a review from sorpaas as a code owner January 29, 2024 12:50
@ahmadkaouk ahmadkaouk requested a review from librelois January 29, 2024 12:50
@ahmadkaouk
Copy link
Copy Markdown
Contributor Author

@sorpaas @boundless-forest can you review this PR please.

@boundless-forest boundless-forest merged commit d3ebb2f into polkadot-evm:master Apr 3, 2024
ahmadkaouk added a commit to moonbeam-foundation/frontier that referenced this pull request Apr 3, 2024
* add precompile clear-storage skeleton

* implement precompile clear storage

* precompile storage cleaner

* mock for testing

* add some rust tests

* add more rust tests

* cover suicided contracts with no storage items

* fix

* fix bugs

* fix precompile

* Update precompile signature

Co-authored-by: Éloïs <c@elo.tf>

* fix precompile signature

* add a limit

* fix limit

* Fix mock

* update precompile

* format Cargo.toml

* rust fmt

* fix clippy warnings

* fix format

* fix tests

* refactor precompile and record weights for clear_suicided_contract

* refactor tests

* fix formatting

* refactor to record cost at the begining

* fix format

* Fix clippy warnings

* update Cargo.lock

* Fix std feature

* update dependencies

* Fix clippy warnings

---------

Co-authored-by: librelois <c@elo.tf>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants